Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NewIR]New ir using kernel registrer type #56789

Merged

Conversation

phlrain
Copy link
Collaborator

@phlrain phlrain commented Aug 30, 2023

PR types

Bug fixes

PR changes

Others

Description

重新提交被revert的pr #56620, 注释部分有冲突的逻辑

对于logical_and的算子,输出的类型是bool类型, 在旧的ProgramDesc中标记为float32类型,导致AOT模块处理异常,新增处理逻辑,从kernel 的output args defs中获取类型,当output args defs内部的数据类型非undefined时,使用output args defs的数据类型

这个本质的问题是 infer Meta把 tensor的dtype 推导逻辑写正确,需要打一个补丁,把这个逻辑补正确

Others

Pcard-67164

@paddle-bot
Copy link

paddle-bot bot commented Aug 30, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot
Copy link

paddle-bot bot commented Aug 30, 2023

✅ This PR's description meets the template requirements!
Please wait for other CI results.

@phlrain phlrain changed the title New ir using kernel registrer type [NewIR]New ir using kernel registrer type Aug 31, 2023
@phlrain phlrain merged commit a34bdb6 into PaddlePaddle:develop Aug 31, 2023
BeingGod pushed a commit to BeingGod/Paddle that referenced this pull request Sep 9, 2023
* update

* fix batch norm grad args def

* fix bug

* fix combine slice bug

* fix slice bug

* update builtin split

* disable using kernel resigter dtype

* polish code

* disable some test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants